Skip to content

[stable-2.0] Make response member in OkHttpMethodBase nullable#381

Merged
tobiasKaminsky merged 1 commit into
stable-2.0from
backport/380/stable-2.0
Jan 14, 2020
Merged

[stable-2.0] Make response member in OkHttpMethodBase nullable#381
tobiasKaminsky merged 1 commit into
stable-2.0from
backport/380/stable-2.0

Conversation

@backportbot-nextcloud

Copy link
Copy Markdown

backport of #380

The class uses non-null lateinit variable that
is not initialized. The member should be of
nullable type.

Fixes nextcloud/android#5073
Fixes nextcloud/android#5102
Fixes nextcloud/android#5100
Fixes nextcloud/android#5142
Fixes nextcloud/android#5156

Signed-off-by: Chris Narkiewicz <hello@ezaquarii.com>
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Lint

Typestable-2.0PR
Warnings00
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings14
Correctness Warnings39
Internationalization Warnings6
Malicious code vulnerability Warnings7
Multithreaded correctness Warnings3
Performance Warnings17
Security Warnings1
Dodgy code Warnings58
Total145

SpotBugs (stable-2.0)

Warning TypeNumber
Bad practice Warnings14
Correctness Warnings39
Internationalization Warnings6
Malicious code vulnerability Warnings7
Multithreaded correctness Warnings3
Performance Warnings17
Security Warnings1
Dodgy code Warnings58
Total145

@codecov

codecov Bot commented Jan 14, 2020

Copy link
Copy Markdown

Codecov Report

Merging #381 into stable-2.0 will decrease coverage by <.01%.
The diff coverage is 29.41%.

@@              Coverage Diff               @@
##           stable-2.0     #381      +/-   ##
==============================================
- Coverage       34.29%   34.29%   -0.01%     
==============================================
  Files             133      133              
  Lines            5724     5727       +3     
  Branches          751      751              
==============================================
+ Hits             1963     1964       +1     
- Misses           3487     3488       +1     
- Partials          274      275       +1
Impacted Files Coverage Δ
.../main/java/com/nextcloud/common/NextcloudClient.kt 30.43% <0%> (ø) ⬆️
...urces/activities/GetActivitiesRemoteOperation.java 36.84% <100%> (ø) ⬆️
...main/java/com/nextcloud/common/OkHttpMethodBase.kt 53.12% <28.57%> (-2.05%) ⬇️

@tobiasKaminsky
tobiasKaminsky merged commit 1b56af7 into stable-2.0 Jan 14, 2020
@delete-merged-branch
delete-merged-branch Bot deleted the backport/380/stable-2.0 branch January 14, 2020 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants